Merge version 1.10.0+ds-1+rpi1 and 1.10.0+ds-2 to produce 1.10.0+ds-2+rpi1

Make fast forward from 1.10.0+ds-1

[dgit --quilt=gbp]

Disable failing xmllint

Gbp-Pq: Name 0008-Disable-failing-xmllint.patch

Ignore test failures due to CPU differences

Gbp-Pq: Name 0008-Ignore-test-failures-due-to-CPU-differences.patch

Add workaround for failing test on i386

Gbp-Pq: Name 0007-Add-workaround-for-failing-test-on-i386.patch

Disable-donation-nagging

Gbp-Pq: Name Disable-donation-nagging

htmldoc-location

Gbp-Pq: Name htmldoc-location

gpsbabelfe_do-not-check-for-newer-version

Disable check for newer versions on start.

Gbp-Pq: Name gpsbabelfe_do-not-check-for-newer-version

gpsbabelfe_opt-in-for-usage-data-reporting

Make statistics reporting an opt-in option.

Gbp-Pq: Name gpsbabelfe_opt-in-for-usage-data-reporting

gpsbabelfe_disable_version_check

Disable version mismatch check.

Gbp-Pq: Name gpsbabelfe_disable_version_check

Update changelog for 1.10.0+ds-2 release

Add patch to disable failing xmllint

Closes: #1113345, #1096761

Merge gpsbabel (1.10.0+ds-1+rpi1) import into refs/heads/workingbranch

Ignore test failures due to CPU differences

Gbp-Pq: Name 0008-Ignore-test-failures-due-to-CPU-differences.patch

Add workaround for failing test on i386

Gbp-Pq: Name 0007-Add-workaround-for-failing-test-on-i386.patch

Disable-donation-nagging

Gbp-Pq: Name Disable-donation-nagging

htmldoc-location

Gbp-Pq: Name htmldoc-location

gpsbabelfe_do-not-check-for-newer-version

Disable check for newer versions on start.

Gbp-Pq: Name gpsbabelfe_do-not-check-for-newer-version

gpsbabelfe_opt-in-for-usage-data-reporting

Make statistics reporting an opt-in option.

Gbp-Pq: Name gpsbabelfe_opt-in-for-usage-data-reporting

gpsbabelfe_disable_version_check

Disable version mismatch check.

Gbp-Pq: Name gpsbabelfe_disable_version_check

gpsbabel (1.10.0+ds-1+rpi1) trixie-staging; urgency=medium

  * Remove armhf from qt6-webengine architecture list.

[dgit import unpatched gpsbabel 1.10.0+ds-1+rpi1]

Import gpsbabel_1.10.0+ds-1+rpi1.debian.tar.xz

[dgit import tarball gpsbabel 1.10.0+ds-1+rpi1 gpsbabel_1.10.0+ds-1+rpi1.debian.tar.xz]

Update changelog for 1.10.0+ds-1 release

Bump policy version (no changes)

Update d/patches

Update d/copyright

Make fast forward from 1.9.0+ds-3

[dgit --quilt=gbp]

Import gpsbabel_1.10.0+ds.orig.tar.xz

[dgit import orig gpsbabel_1.10.0+ds.orig.tar.xz]

Ignore test failures due to CPU differences

Gbp-Pq: Name 0008-Ignore-test-failures-due-to-CPU-differences.patch

Add workaround for failing test on i386

Gbp-Pq: Name 0007-Add-workaround-for-failing-test-on-i386.patch

Disable-donation-nagging

Gbp-Pq: Name Disable-donation-nagging

htmldoc-location

Gbp-Pq: Name htmldoc-location

gpsbabelfe_do-not-check-for-newer-version

Disable check for newer versions on start.

Gbp-Pq: Name gpsbabelfe_do-not-check-for-newer-version

gpsbabelfe_opt-in-for-usage-data-reporting

Make statistics reporting an opt-in option.

Gbp-Pq: Name gpsbabelfe_opt-in-for-usage-data-reporting

gpsbabelfe_disable_version_check

Disable version mismatch check.

Gbp-Pq: Name gpsbabelfe_disable_version_check

Update upstream source from tag 'upstream/1.10.0+ds'

Update to upstream version '1.10.0+ds'
with Debian dir 4dadd5fb004597a6405d79eae4b0929723e2add6

New upstream version 1.10.0+ds

Drop qtci from Files-Excluded (dropped upstream)

release candidate 1.10.0 (#1390)

update dev archive patch for recent PR merge. (#1389)

cleanup GUI special member functions (#1387)

* cleanup gui special member functions

These tidy checks had a large impact:
cppcoreguidelines-prefer-member-initializer
modernize-use-default-member-init
modernize-use-equals-default

Some instances of other tidy issues were resolved:
clang-analyzer-optin.cplusplus.VirtualCall
cppcoreguidelines-special-member-functions
readability-inconsistent-declaration-parameter-name
readability-named-parameter
and others.

Overall we have 1334 fewer tidy checks on our code base, a
reduction of 15% of the total!

The user defined dtor for class SettingGroup was eliminated
by using std::vector<std::unique_ptr<VarSetting>> instead of a
QList of raw pointers.
The user defined dtor for class FilterWidget was eliminated
by using std::vector<std::unique_ptr<FilterOption>> instead of a
QList of raw pointers.

fix clang-analyzer-core.NullDereference ... (#1388)

which was due clang making false assumptions.
But I think the modified code is easier for both humans and clang to
understand.

fix booboo caught by clang (#1386)

Refactor GUI map preview (#1385)

* refactor map preview

"Show Only This ..." will pan to the selected waypoint or frame the
selected track or route.
"Show All ..." will recenter the map and reset the bounds.

delete map dialog copy btn that was un-implemented. (#1384)

use iwyu to regenerate gui includes. (#1382)

delete obsolete slot. (#1381)

this should have been deleted in #681
(8bfb2c52f2731b0a01ebfe6d840fb9d19ba6f51c)

fix tidy modernize-use-override (#1380)

fix mysterious lost message (#1370)

manually trigger attestations (#1379)

* use dispatch parameter to include attestation.

* restrict attestation to release flavor

add Fedora 41 to CI. (#1378)

need write permission for upload tool (#1377)

Attestation (#1376)

* try attestation on windows continuous releases.

* add attestation for macos, linux continuous releases.

Get clazy working again (#1375)

delete obsolete files. (#1374)

* delete obsolete files.

* remove ziparchive which was never used.

fix markdownlint warnings. (#1373)

update precompiled header list. (#1372)

remove commented out tpo code from 2012. (#1371)

Use QtLogging Message Pattern to replace MYNAME (#1367)

* use qlogging message pattern to replace MYNAME.

* first cut at removal of MYNAME.

* add message type support

and massage some logging with internal newlines.

* introduce DebugLog class to transform printf ...

style character output into lines suitable for logging.

* handle debug printing in garmin_gpi

including removal of POSIX printf arguement reordring

* use DebugLog with lowranceusr

* use DebugLog in dg100

* fix logging in garmin_txt

* use DebugLog in exif

* use logging for validate.

* fix includes

* use DebugLog in igc

* use logging in kml

* use logging in garmin

* use DebugLog in mtk.

* use DebugLog in tpo

* use logging in trackfilter

* use DebugLog for v900

* use DebugLog in skytraq

* use logging with nmea.

* fix dg100, update references, chop extra newlines.

* appease MSVC C2131

* fix garmin_gpi address debug messages.

and direct vecs debug output to debug msg.

* fix comparison with testo -D 9 igc format

* match globalsat sport ws debug messages

* match ws on fatal()

* update references for option->stderr and

no extra newline on fatal.

* ws

* don't send empty string to QDebug.

* drop another empty string from logging.

* cleanup fatal, flush is done in message handler.

* move conditional logging check earlier.

* make member functions static

* fix debug build.

* eliminate some QString<->QByteArray round trips.

* purse QStrings from fatal routines.

Besides an efficiency hit any variadic argument strings passed to these
routines should already be in local8Bit.

* rename logging functions for easier script editing.

thanks clion.

* add macro gbLogCStr for use with our logging.

All but one line was changed by sed (ok, the sed script took
a few tries.)
This allows us to change the encoding passed to our printf style
logging routines in one place.

* change logging back to using QStrings.

QtMessageHandler operates on QStrings.

* manual rename logging routines in windows only files.

* use global gbLog instead of DebugLog instances.

* convert all gb printflike logging to accumulate lines.

* use gbDebug instead of gbLog.  They are equivalent.

* remove unecessary choping.

* fix getLogString, flush fatal log stream.

* terminate gbFatal format strings with a newline.

Since we flush this isn't necessary, but let's be consistent.

* terminate gbWarning format strings.

* switch to our own MessageHandler for legacy logging.

We don't delay logging until we have a complete line, instead
we output any bits as we go.  We only format the log message if
the last output for the first message or if the last output
character was a newline.

* rename LegacyLog components.

* update fatal.cc includes

* use same stream in waypt_status_disp & termination in waypt_disp_session.

stderr is a text stream which should translate '\n' -> '\r\n' on windows
automatically.

* review catches, mostly in comments or dead code.

hide logging member variables, use references for stream op. (#1369)

* hide logging member variables, use references for stream op.

* match parameter naming.

* longer names.

ws reference update (#1368)

add new teset to cmake (#1366)

clang tidy fixes (#1365)

* fix tidy warings related to passing const container.

readability-avoid-const-params-in-decls
performance-unnecessary-value-param

* tidy readability-inconsistent-declaration-parameter-name

* tidy cppcoreguidelines-prefer-member-initializer

* tidy readability-redundant-member-init

* tidy modernize-loop-convert

* tidy modernize-type-traits

* tidy readability-named-parameter

move initialization of OptionInt/Double parameters to constructor (#1364)

* move init of OptionInt/Double parms to ctor

except for id, which isn't known by formats/filters.
Also, move validation of OptionInt/Double strings to Option
class, where it can be maintained in parallel with the conversion
routines.

* init Option id in init_vec, init_filter_vec.

also
fix -Wclazy-unused-non-trivial-variable.
add test of Option error detection.

protect invariance of OptionInt, OptionDouble (#1363)

* protect invariance of OptionInt, OptionDouble

we want to ensure the source string always corresponds to the
results of conversion.

* fix spelling in this mornings grammar fix!

* remove obsolete include

remove obsolete comparison to option name (#1362)

minor cleanups (#1361)

* resolve minor nits

* grammar

introduce OptionString, (#1352)

* introduce OptionString,

a replacement for OptionCString. Formats and filters can be
manually converted, with the potential to eliminate c character
string usage.
OptionCString::get() usages can often use an implicit cast from
OptionString instead, although the usage of get() is still valid.

* more OptionCString -> OptionString conversion

* use custom conversion routines.

this introduces OptionString::toInt and OptionString::toDouble,
which enforce error checking on conversions.

* convert more formats to OptionString

* convert gdb to OptionString

* convert xcsv, unicsv to OptionString

* convert html, text to OptionString

* convert ozi to OptionString

* convert mtk to OptionString

* convert globalsat to OptionString

* convert exif to OptionString

* convert lowranceusr to OptionString

* convert igc to OptionString.

note the error checking on timeadj is now done by OptionString::toInt.

* prove igc timeadj works with ints.

* convert shape to OptionString

* convert garmin_gpi to OptionString

* consolidate integer/double parse routines.

* convert subrip to OptionString

* convert garmin_txt to OptionString

* enhance OptionString conversions.

overloads, as opposed to default parameters, allow tools to find differen usages.

improved OptionString error messages with module and argstring information.

* enhance argtype ...

to indicate integer base and if trailing data is allowed.
Use these fields to check and convert integer/doubles in Vecs, fataling on errors.

* allow trailng data with parse_distance, parse_speed.

and when necessary convert these from ARGTYPE_STRING to ARGTYPE_FLOAT.

* fix typo, update refs

* convert garmin to OptionString

* convert skytraq to OptionString

* Introduce OptionInt & OptionDouble.

These must be used with ARGTYPE_INT and ARGTYPE_FLOAT.
The data is checked and converted in Vecs, and the results may
be retrieved by the user.

* convert almost all ARGTYPE_INT to OptionInt.

* final conversion to OptionInt

* mark TODO done.

* correct include comments for greps sake.

* delete complete TODO comments.

* whitespace

* use consistent error messags regarding options.

i.e. "module(option): "

* delete unused Option method type

* use ARGTYPE_STRING if trailing data is allowed.

The GUI validator used with ARGTYPE_INT and ARGTYPE_FLOAT will
reject input with any trailing data.

The CLI, including Vecs::assign_option, keys off the Option subclass now,
not the ARGTYPE.

Note that OptionInt and OptionDouble still allow trailing data.  If
trailing data is allowed with these classes we just pair these with
ARGTYPE_STRING which controls validation in the GUI.

* drive arg validation from Option classes.

* simplify use of isEmpty

delete unused macro
delete extraneous file

* non member operator== with parameters const QString& not a candidate,

parameters are of class OptionString.

use correct compare in enhanced garmin_gpi test (#1360)

enhance subrip, garmin_gpi tests (#1359)

* add test for garmin gpi speed/proximity options.

* enhance subrip test

restore lowranceusr truncation in bytes (#1358)

as opposed to QChars.

Qtize garmin_tables (#1357)

Qtize rgbcolors. (#1356)

* Qtize rgbcolors.

* use big stick to validate hex color codes

delete unimplemented options (#1355)

* delete globalsat_sport track option

which was never implemented.

* delete ancient geo nuke_placer option

which hasn't worked since ~1.4.4

use consistent unit suffixes. (#1353)

* use consistent unit suffixes.

This is a user visible change. The suffixes for the distance
option of the arcdist, interpolate, radius and position filters
and the add option of the height filter are different.

* drop unused includes

* whitespace

* terminate fatal messages.

* change simplify to process distances in meters.

This simplifies option value parsing.
There is no user visible difference.

* define MYNAME in cc not h.

refactor option handling (#1351)

* rough in Option class

* testo passes

* rough in OptionBool

* more OptionBool use

* more OptionBool

* fix isEmpty

* store qstrings

* dont return const

* rename

* convert a few lost ARGTYPE_BOOLs to OptionBool.

* utilize avalable QString for options.

* obsolete argvalptr

* Option is abstract class

* eliminate special handling of ARGTYPE_BOOL without default in vecs.

* fix #982 ...

garmin, ozi, xcsv mkshort options whitespace_ok, must_unique broken since 2006.

* test to make sure OptionBool iff ARGTYPE_BOOL

* use OptionBool for all ARGTYPE_BOOL options.

* correct OptionBool type.

* delete unused parameterized Option ctors.

* format and comment option.h

* nodiscard for class Option.

* vecs nullptr checks

* have Vecs::validate_args check argval isn't nullptr.

* catch up includes

* clarify comment

* fix introduced bug caught by tidy.

Revert "fix an annoying Wstringop-truncation warning. (#1348)" (#1349)

This reverts commit a4dec0acfbeb6d93a9a73e3673da983bfdd7511b.

fix an annoying Wstringop-truncation warning. (#1348)

* fix an annoying Wstringop-truncation warning.

while the warning is fixed we can still index outside of the
gps_categories array :(

* solved stringop-truncation w/o format macro ...

constants, rely on integer promotion.
Also, guard against devices with more than 16 categories should
they exist.

further qstringification of osm (#1347)

* clean up code issues in osm.

including repeated char* -> QString conversions.

* add include

* qstringification

* remove cstring from osm

* fiddle with osm creator.

prepare for Qt 6.8 deprecations (#1346)

* fix Qt 6.8.0 deprecation warnings.

Also, the exif writer now prefers to establish a timestamp from
GPS_IFD_TAG_DATESTAMP/GPS_IFD_TAG_TIMESTAMP if they exist, as the
exif reader does.

* avoid installing *.debug_information modules

which appeared with 6.7.0

* centralize decision on lightweight time zones.

* correct sort order for includes, duplicates

* remove debug statement

* remove const from declaration.

* simplify lightweight timezone usage via

qt version depependent defined constants.

* fix for QTimeZone not a QLiteralType

* cleanup lightweight time constants.

* enhance exif offset test

* add missing reference

* fix test comment

* revert changes to exif writer regarding gps tags.

I beleive the original intent of the writer is to create the gps tags
from a waypoint.

* update qt modules

* qt 6.8 work

1. allow builds to succeed without webengine by default.  This is
needed by CI until aqt catches up.
2. macos minos change for 6.8
3. add macos 6.8.0 to CI, remove macos-12 build.

* avoid Wunused-const-variable

* add 6.8.0 to ci for windows

use containers for xml_init lists. (#1345)

use container for unicsv fields def. (#1344)

and cleanup an extraneous break from #1164.

catch gdb debug up with gbfile changes. (#1343)

* catch gdb debug up with gbfile changes.

* use "if constexpr()" for GDB_DEBUG to detect errors

even if not compiling for debug.

more QStrings in gbfile (#1342)

rewrite strip_nastyhtml, strip_html in Qt (#1341)

* rewrite strip_html with QString.

* rewrite strip_nasty_html in Qt.

and actually produce valid html:
1. the replacement for "<body>", "<!   >", is invalid.
2. leaving an html tag in causes the html format output to be invalid.

* cleanup comment xstrdup

* use regex for strip_html

* strip_html deletes other tags

* fix strip_html img tag handling

* Revert "fix strip_html img tag handling"

This reverts commit b0440f7e3729909fd51ecbbacdd7a7e4aa5d9198.

* Revert "strip_html deletes other tags"

This reverts commit 40fe2ef0153d8c2e6d53432b9184f8acffbab2ef.

* Revert "use regex for strip_html"

This reverts commit 677da95a77b335bfa686a35fc33e862f8b50f053.

* implement strip_html using QRegularExpressionMatchIterator.

* a little cleanup

* remove obsolete include

* take care to distinguish tags with common roots

like p, param, pre.

* suppress InvalidReads in qhash.

These are known to occur per comment in qhash.cpp.

* supress qhash false positive with libqt6core6/jammy-updates,now 6.2.4+dfsg-2ubuntu1.1 amd64

* suppress vg warnings on noble (intermittant).

* install qt core dbgsyms for valgrind suppression.

* kill space preceding newline when stripping html.

* use modernize-raw-string-literal

* valgrind suppressions for f40

* add symbols for fedora valgrind suppression.

remove obsolete mkshort test. (#1340)

The macro TEST_MKSHORT hasn't existed for some time.

fixes for pr1238. (#1339)

shapelib is handled in shapelib.cmake.
CSVFMTS_ENABLED is now unreferenced.

Remove XCSV_ENABLED, MAXIMAL_ENABLED, MINIMAL_ENABLED (#1238)

Remove XCSV_ENABLED, MAXIMAL_ENABLED, MINIMAL_ENABLED
---------

Co-authored-by: Robert Lipe <robertlipe@gpsbabel.org>

Update gpsbabel_man.xml (#1169)

1. add dash, else looks like old tar(1) command.
2. Fix grammar.

Co-authored-by: Robert Lipe <robertlipe@users.noreply.github.com>

bring back See You support (#1280)

I think someone was waiting for me to say no. Sometimes, "good enough" (for users with no representation here...) seems to be.
Co-authored-by: Robert Lipe <robertlipe@users.noreply.github.com>

Various modernizations in humminbird. (#1100)

* Various modernizations in humminbird.

- Make humminbird_icons[] a QStringList.
- I don't think the icon finder ever actually worked the way it was described in the
comments.
- Comment fix in exif
- Kill xstrndup (yay)

modernize some dynamic memory usage. (#1334)

* modernize some dynamic memory usage.

* refactor convert_human_[date|time]_format

* convert convert_human_*_format to Qt.

* fix include comment

* use back instead of *rbegin to reference last QChar in QString

use persistant docker container for GUI (#1338)

update packages for jammy, noble. (#1337)

create docker images with historical releases. (#1336)

* create docker images with historical releases.

* add scripts to automate image creation

* enhance test script to avoid cwd dependencies.

* move 1.7.0 base from focal to jammy

* make sure to clean up container

* make sure container is cleaned up.

* fix varaible use in arithmetic context

* delete unused archive from images.

* don't confuse our zlib and system zlib

* use system zlib shapelib for 1.8.0, 1.9.0

* add script to run GUI

* fix up 1.5.x plugins.

these were probematic for 1.5.4, previous releases ran ok with webkit.
translations don't work for 1.5.x

* shellcheck

* enhance GUI run script to forward LANG

* add bleeding edge image.

Merge pull request #1335 from GPSBabel/tidymore

clang tidy fixes

fix tidy modernize-use-nullptr

fix tidy readability-duplicate-include

fix tidy modernize-raw-string-literal

fix tidy readability-avoid-const-params-in-decls